Skip to content

removed curation list view#2269

Merged
ragnep merged 1 commit intomainfrom
remove-curation-list
Apr 15, 2026
Merged

removed curation list view#2269
ragnep merged 1 commit intomainfrom
remove-curation-list

Conversation

@ragnep
Copy link
Copy Markdown
Contributor

@ragnep ragnep commented Apr 15, 2026 •
edited by coderabbitai Bot
Loading

Summary by CodeRabbit

  • Refactor
    • Streamlined the user profile curation interface by removing the view mode toggle control. Users previously had the option to switch between masonry and list display layouts when viewing their profile curations, but now the content displays exclusively in masonry view. This change simplifies the interface and reduces unnecessary UI elements.

@ragnep
removed list view
8616884 
Signed-off-by: ragnep <ragneinfo@gmail.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026 •
edited
Loading

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 99e31f0c-66aa-4a21-ba73-406e915d9a78

📥 Commits

Reviewing files that changed from the base of the PR and between daf6959 and 8616884.

📒 Files selected for processing (2)
  • components/user/waves/UserPageProfileWave.tsx
  • hooks/useProfileCurationViewMode.ts
💤 Files with no reviewable changes (1)
  • hooks/useProfileCurationViewMode.ts
📝 Walkthrough

Walkthrough

The changes remove the profile curation view mode toggle feature, eliminating the ability to switch between "masonry" and "list" view modes. The curation now always renders in masonry mode, and the associated hook managing this state is deleted.

Changes

Cohort / File(s) Summary
View Mode Toggle Removal
components/user/waves/UserPageProfileWave.tsx, hooks/useProfileCurationViewMode.ts		 Removed ProfileCurationViewToggle component and useProfileCurationViewMode hook; simplified curation rendering to always use masonry mode instead of conditionally switching between masonry and list views.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Suggested reviewers

  • simo6529
  • prxt6529

Poem

🐰 A toggle once danced in the UI so grand,
But masonry alone is now the land,
No more switching views with a click and a swirl,
Just clean, simple walls of content to unfurl! 🏛️

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'removed curation list view' accurately and concisely summarizes the main change: eliminating the list view toggle and related functionality from the curation display, leaving only masonry view.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch remove-curation-list

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Apr 15, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@ragnep ragnep merged commit 04e109b into main Apr 15, 2026
8 checks passed
@ragnep ragnep deleted the remove-curation-list branch April 15, 2026 05:50
@coderabbitai coderabbitai Bot mentioned this pull request Apr 15, 2026
Open
@coderabbitai coderabbitai Bot mentioned this pull request Apr 28, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@GelatoGenesis GelatoGenesis GelatoGenesis approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ragnep @GelatoGenesis